Skip to content

Try to avoid matching double space in tables. - #328

Closed
garloff wants to merge 1 commit into
mainfrom
fix/dbl-space-rule-no-tables
Closed

Try to avoid matching double space in tables.#328
garloff wants to merge 1 commit into
mainfrom
fix/dbl-space-rule-no-tables

Conversation

@garloff

@garloff garloff commented Nov 26, 2025

Copy link
Copy Markdown
Member

This fixes the markdownlint rule.

This fixes the markdownlint rule.

Signed-off-by: Kurt Garloff <kurt@garloff.de>
@garloff garloff self-assigned this Nov 26, 2025
@garloff garloff added the bug Something isn't working label Nov 26, 2025
@garloff

garloff commented Nov 26, 2025

Copy link
Copy Markdown
Member Author

This fixes #329.
Whether it's the correct fix, I don't know.

  • Ideally, markdownlint-rule-search-replace would have a skipTable (or a table searchScope) setting, but I did not find such a thing
  • My change might result in only the first double-space in a line to be fixed (if we're using the rule for fixing and not erroring out).

@garloff
garloff requested review from fkr and maxwolfs November 26, 2025 15:33
@garloff garloff added the documentation Improvements or additions to documentation label Nov 26, 2025
@garloff

garloff commented Nov 26, 2025

Copy link
Copy Markdown
Member Author

Alternative approach: #330.

@garloff garloff added the work in progress Pull requests that are work in progress, do not merge them label Nov 26, 2025
@garloff

garloff commented Nov 27, 2025

Copy link
Copy Markdown
Member Author

Preferring #330 for now.

@garloff garloff closed this Nov 27, 2025
@github-project-automation github-project-automation Bot moved this from Backlog to Done in Sovereign Cloud Stack Nov 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working documentation Improvements or additions to documentation work in progress Pull requests that are work in progress, do not merge them

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant